Skip to content

feat(telemetry link): Onboarding TelemetryLink#1456

Open
ozanichkovsky wants to merge 31 commits into
stackitcloud:mainfrom
ozanichkovsky:feature/telemetrylink
Open

feat(telemetry link): Onboarding TelemetryLink#1456
ozanichkovsky wants to merge 31 commits into
stackitcloud:mainfrom
ozanichkovsky:feature/telemetrylink

Conversation

@ozanichkovsky
Copy link
Copy Markdown
Contributor

@ozanichkovsky ozanichkovsky commented May 11, 2026

Description

TelemetryLink service resources and datasources.

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@ozanichkovsky ozanichkovsky requested a review from a team as a code owner May 11, 2026 20:00
@ozanichkovsky ozanichkovsky changed the title feat: TelemetryLink resources and datasources feat(telemetry link): Onboarding TelemetryLink May 11, 2026
Fyusel
Fyusel reviewed May 12, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Fyusel Fyusel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some first review comments, need to have a deeper look

Comment thread examples/data-sources/stackit_telemetrylink_link/data-source.tf Outdated
Comment thread examples/resources/stackit_telemetrylink_link/resource.tf Outdated
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/datasource.go
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Fyusel
Fyusel reviewed May 12, 2026
View reviewed changes
Comment thread stackit/internal/services/telemetrylink/telemetrylink_acc_test.go Outdated
Fyusel
Fyusel reviewed May 15, 2026
View reviewed changes
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/telemetrylink_acc_test.go Outdated
Comment thread stackit/internal/services/telemetrylink/telemetrylink_acc_test.go Outdated
@h3adex
Copy link
Copy Markdown
Contributor

h3adex commented May 27, 2026

docs are missing ^

Fyusel
Fyusel reviewed May 29, 2026
View reviewed changes
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
Fyusel
Fyusel reviewed May 29, 2026
View reviewed changes
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/datasource.go Outdated
Comment thread stackit/internal/services/telemetrylink/link/resource.go Outdated
@Fyusel
Copy link
Copy Markdown
Contributor

Fyusel commented May 29, 2026

go mod tidy is needed

@Fyusel
Copy link
Copy Markdown
Contributor

Fyusel commented May 29, 2026

Acc tests are failing

@ozanichkovsky
Copy link
Copy Markdown
Contributor Author

ozanichkovsky commented May 29, 2026

Can't generate docs because I get an error.

Error executing command: unable to generate website: error rendering static website: unable to render templated website to static markdown: unable to render data source template "data-sources/affinity_group.md.tmpl": unable to execute template: template: resourceTemplate:34:6: executing "resourceTemplate" at <.HasExamples>: can't evaluate field HasExamples in type struct { Type string; Name string; Description string; HasExample bool; ExampleFile string; HasImport bool; ImportFile string; ProviderName string; ProviderShortName string; SchemaMarkdown string; RenderedProviderName string }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fyusel Fyusel Fyusel left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

has internal tracking issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ozanichkovsky @h3adex @Fyusel @GokceGK